-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Fix the script that generates TFJava doc #2332
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Hi, checking again if we can get some help, since updating the documentation on the website is pretty much the last thing remaining for us before publishing TF Java 1.0.0 cc\ @MarkDaoust |
Hi @MarkDaoust and @markmcd , any update on this please? It is a bit frustrating that we can't update the documentation on the website and some of our users think our project is inactive (while it is not). |
Can you mark the PR not in draft mode? |
The PR probably doesn't want to be merged in it's current state, as we don't have enough insight into how the process is run internally to generate the docs. What we need is to discuss with the docs team how to get them to pull our documentation over from tensorflow/java, then this updated script can be properly incorporated. |
Hey @mihaimaruseac , I've marked the PR as non-draft, any chance we can solve this soon? Thank you |
Hi Karl, the TensorFlow site has fewer people/resources devoted to it now. The old system where we were in the loop for doc updates probably isn't feasible any more. What if we replaced the old doc pages on tensorflow.org with a redirect to your teams docs, in a location of your choice, that you control? that could resolve this for the future. |
If you're happy for it not to be themed as well as the TF documentation is, and Google is fine with us running a GitHub Pages instance out of the |
PLEASE DO NOT MERGE THIS PR AND READ THE TEXT BELOW
Hi, we (SIG-JVM) would like the TensorFlow Java API docs to be updated on the TensorFlow website.
We are sending you this script used to generated the Javadoc files, but unless the process has changed, the script should be retrieved from our own repo under that location. So one main reason to create this PR was also to find a way to reach out to you to share our intent (btw is there another communication channel that we can use other than GitHub PRs?). It is an update from the one @MarkDaoust originally contributed to our repo.
In addition, we have updated the install instructions that you can retrieve here.
We are happy to refactor this process as well, we are not sure anymore where that script should live.